Skip to content

Conversation

@pxpm
Copy link
Contributor

@pxpm pxpm commented Oct 10, 2025

Add demo for Modal Form Component

To use all branches you want to test, do:

# in demo directory
gh pr checkout 721

# pro
cd vendor/backpack/pro
gh pr checkout 329

# crud
cd ../../..
cd vendor/backpack/crud
gh pr checkout 5863

# back
cd ../../..
php artisan optimize:clear
php artisan basset:clear
php artisan basset:cache

@pxpm pxpm force-pushed the add-modal-form-examples branch from 75db95e to e8055d5 Compare October 24, 2025 15:05
@pxpm pxpm force-pushed the add-modal-form-examples branch from e8055d5 to 2b3d673 Compare October 24, 2025 15:10
…Backpack/demo into add-modal-form-examples

# Conflicts:
#	resources/views/admin/partials/dataform-examples.blade.php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i go to "New in v7", scroll down to "Edit Skill", clear the name input and hit SAVE - I get an error from InvoiceCrudController - why?! I'm trying to edit a skill here, not an invoice - here's the video https://share.cleanshot.com/kdhq0tBx - and here's the error stack https://flareapp.io/share/V7j8V1Xm

<div class="card-body bg-light">
<div class="card-body">

<x-backpack::dataform
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here's what I did:

  • went to MonsterCrudController, edited a monster in a modal;
  • went on "New in v7" page, quick added a tag;
  • expected to get redirected to the Tag List, but instead I got redirected to the Monster List;
CleanShot.2025-10-27.at.11.17.07.mp4

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I then go to "New in v7", in Create Invoice, and add an invoice from there... I get the same error:

CleanShot.2025-10-29.at.18.32.47.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants